Skip to content

Purge ban now shows that user was purge banned within infraction message.#1453

Merged
Xithrius merged 3 commits into
masterfrom
ban-vs-pban
Mar 8, 2021
Merged

Purge ban now shows that user was purge banned within infraction message.#1453
Xithrius merged 3 commits into
masterfrom
ban-vs-pban

Conversation

@Xithrius
Copy link
Copy Markdown
Contributor

@Xithrius Xithrius commented Mar 8, 2021

Closes #1412

Just a small QOL change. When a user is purge banned, the feedback message shows that the purge was applied.

The difference between a regular ban and a purge ban:
image

@Xithrius Xithrius marked this pull request as draft March 8, 2021 11:23
@Xithrius Xithrius added a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) p: 2 - normal Normal Priority t: enhancement Changes or improvements to existing features labels Mar 8, 2021
@Xithrius Xithrius marked this pull request as ready for review March 8, 2021 11:38
Copy link
Copy Markdown
Contributor

@Shivansh-007 Shivansh-007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Looks Good To Me!

Copy link
Copy Markdown
Contributor

@HassanAbouelela HassanAbouelela left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simple, and clean. Thanks

Copy link
Copy Markdown
Contributor

@Akarys42 Akarys42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@Xithrius Xithrius merged commit 0d6925e into master Mar 8, 2021
@Xithrius Xithrius deleted the ban-vs-pban branch March 8, 2021 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: backend Related to internal functionality and utilities (error_handler, logging, security, utils and core) p: 2 - normal Normal Priority t: enhancement Changes or improvements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show purgebanned in the feedback message of pban

4 participants